-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Github actions #55
base: master
Are you sure you want to change the base?
Github actions #55
Conversation
tizoc
commented
Mar 28, 2021
•
edited
Loading
edited
- Test build
- Produce and publish release binaries
84da33b
to
1fa1062
Compare
For some reason the tests for the CLisp build fail on Linux, but I am not sure why. |
7821e42
to
2933f9b
Compare
So, the CLisp versions from Ubuntu Groovy, Ubuntu Bionic, and Ubuntu Xenial all behave differently, and the first two are broken. I will disable that for now because I don't have the time or energy to deal with it at the moment. |
Changelog here from what seems to have changed on CLisp by the Debian maintainers: https://metadata.ftp-master.debian.org/changelogs//main/c/clisp/clisp_2.49.20180218+really2.49.92-3_changelog |
This fixes the REPL on the version provided by Ubuntu bionic
Taking CLisp off the default build list is fine I think. SBCL the port, you know. By the way, CLISP 2.49 (2010-07-07) has always worked for me. |
@rkoeninger same here, but turns out that newer versions of Ubuntu (and I guess Debian), use updated versions which don't behave the same as 2.49 (and they don't even seem to behave the same between Ubuntu versions). It is weird that Clisp's latest official release was more than 10 years ago, but I see a bunch of changes up to at least 2018, but no new releases. Btw, I agree that taking Clisp out from CI is fine (but will probably update the Github Actions script and leave the Makefile with the "all" target as it was). I spent the little time I had this last weekend fighting with those issues and I wasn't able to finish the "release" pipeline, will probably do so this weekend. |